Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.46%. Comparing base (c4394bb) to head (fe7d138).
Report is 2 commits behind head on main.

@@            Coverage Diff             @@
##             main     #105      +/-   ##
==========================================
+ Coverage   91.40%   91.46%   +0.05%     
==========================================
  Files          22       22              
  Lines        2316     2331      +15     
==========================================
+ Hits         2117     2132      +15     
  Misses        199      199              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

This raised a couple of issues:

• A SemidiscretizationHyperbolic constructor had been removed and compute_coefficients got a new argument in
  Use Adapt.jl to change storage and element type Trixi.jl#2212

• The default wave speed function for Local Lax-Friedrichs was changed to max_abs_speed instead of max_abs_speed_naive: Change default wave speed for Local Lax-Friedrichs Trixi.jl#2458

• I added max_abs_speed for ShallowWaterEquations3D in analogy to what is done for the Euler equations. If this is ok, we probably still need to adjust reference values.

• For AbstractCovariantShallowWaterEquations2D and CovariantLinearAdvectionEquation2D I just renamed max_abs_speed_naive to max_abs_speed as the implementation did not seem to be "naive". Please double-check!

• Unrelated to Trixi, DocumenterInterLinks.jl introduced opt-in for automatic resolution of references to external sources:
  https://github.com/JuliaDocs/DocumenterInterLinks.jl/releases/tag/v1.1.0

Could you please check the CI reference values so that tests pass?

* The default wave speed function for Local Lax-Friedrichs was changed to `max_abs_speed` instead of `max_abs_speed_naive`: [Change default wave speed for Local Lax-Friedrichs Trixi.jl#2458](https://github.com/trixi-framework/Trixi.jl/pull/2458)

* I added `max_abs_speed` for `ShallowWaterEquations3D` in analogy to what is done for the Euler equations. If this is ok, we probably still need to adjust reference values.

This looks valid, see also trixi-framework/TrixiShallowWater.jl#65